This repository was archived by the owner on Feb 22, 2023. It is now read-only.
[connectivity_for_web] Migration to null-safety.#3652
Merged
fluttergithubbot merged 7 commits intoflutter:masterfrom Mar 2, 2021
Merged
[connectivity_for_web] Migration to null-safety.#3652fluttergithubbot merged 7 commits intoflutter:masterfrom
fluttergithubbot merged 7 commits intoflutter:masterfrom
Conversation
… instead of an internal subscription to another stream (which was hard to cleanup)
…ream, to prevent multiple attachments on hot reload.
ditman
commented
Mar 2, 2021
| /// The constructor of the plugin. | ||
| NetworkInformationApiConnectivityPlugin() | ||
| : this.withConnection(html.window.navigator.connection); | ||
| : this.withConnection(html.window.navigator.connection!); |
Member
Author
There was a problem hiding this comment.
This is ensured by the isSupported method that is used before instantiating this plugin class.
Contributor
There was a problem hiding this comment.
Seems worth documenting that requirement in the constructor comment.
stuartmorgan-g
approved these changes
Mar 2, 2021
| } | ||
|
|
||
| StreamController<ConnectivityResult> _connectivityResult; | ||
| late StreamController<ConnectivityResult> _connectivityResult; |
Contributor
There was a problem hiding this comment.
Huh, I thought you couldn't null-check a late value (and that lazy initialization like the below needed ? as a result).
Member
Author
There was a problem hiding this comment.
You're right, let me give this a test!
Member
Author
There was a problem hiding this comment.
This was indeed crashy. Fixed. This file needs a test (mock window + navigator), I'll add one after this lands.
| /// The constructor of the plugin. | ||
| NetworkInformationApiConnectivityPlugin() | ||
| : this.withConnection(html.window.navigator.connection); | ||
| : this.withConnection(html.window.navigator.connection!); |
Contributor
There was a problem hiding this comment.
Seems worth documenting that requirement in the constructor comment.
Member
Author
|
The requirement of the 'connection' API is documented in the README: https://github.com/flutter/plugins/tree/master/packages/connectivity/connectivity_for_web#limitations-on-the-web-platform |
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Mar 2, 2021
fluttergithubbot
pushed a commit
to flutter/flutter
that referenced
this pull request
Mar 2, 2021
NickalasB
added a commit
to NickalasB/plugins
that referenced
this pull request
Mar 3, 2021
* master: Adopt Xcode 12 for podspec lints (flutter#3653) Run static analyzer during xctest (flutter#3667) [google_maps_flutter_web] update min flutter sdk version to 1.20.0 (flutter#3662) [image_picker] Run CocoaPods iOS tests in RunnerUITests target (flutter#3663) [webview_flutter] Run CocoaPods iOS tests in RunnerUITests target (flutter#3664) [device_info] Enable NNBD for unit test (flutter#3658) remove unused plugin (flutter#3661) [android_intent] move unit test to nullsafety (flutter#3659) [url_launcher] Migrate unit tests to NNBD (flutter#3657) [share] Migrate unit tests to null-safety. (flutter#3660) [connectivity_for_web] Migration to null-safety. (flutter#3652) [camera] Stable release for null safety. (flutter#3641) [in_app_purchase] fix plugin version (flutter#3654) Move plugin tool tests over (flutter#3606) [in_app_purchase] migrate playing billing library to v3 (flutter#3636) Update plugin_platform_interface min version (flutter#3650) # Conflicts: # packages/webview_flutter/CHANGELOG.md # packages/webview_flutter/example/ios/Runner.xcodeproj/project.pbxproj
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change migrates the package to null safety, and moves its (very obsolete) tests to the new integration_test style.
The tests used to use mockito, but those mocks have been replaced by Fake implementations.
Tests
Pre-launch Checklist
[shared_preferences]///).If you need help, consider asking for advice on the #hackers-new channel on Discord.